Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Traducido observables.md a español fix #214 #258

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jaov
Copy link

@jaov jaov commented Oct 2, 2020

Se mantiene el uso de streams para referirse a flujos ya que se conocen más por su nombre en inglés, incluso en la comunidad hispana.

Lista de Verificación del PR

Comprueba si tu PR cumple los siguientes requisitos:

Resolviendo Issue #214

  • El mensaje de commit esta conforme con nuestras reglas
  • Probe los cambios que agregué (arreglo de bugs / funcionalidades)
  • Revisé previamente las traducciones o cambios de contenido
  • Consulté el diccionario de términos en español
  • He creado dos archivos con la extensión correspondiente(.en.md para el archivo en inglés y .md para el Archivo en español)
  • La PR fue enlazada con el issue correspondiente

Tipo de PR

¿Qué tipo de cambio introduce este PR?

  • Cambios en el contenido de la documentación
  • Otro... Por favor describe la:

¿Cuál es el comportamiento actual?

Archivo está en inglés

¿Cuál es el nuevo comportamiento?

Archivo de inglés traducido al español

Se mantiene el uso de streams para referirse a flujos ya que se conocen más por su nombre en inglés, incluso en la comunidad hispana.
@CLAassistant
Copy link

CLAassistant commented Oct 2, 2020

CLA assistant check
All committers have signed the CLA.

@AntonioCardenas AntonioCardenas requested a review from a team October 2, 2020 18:34
@gustavguez
Copy link
Member

@jaov Gracias por colaborar! debes firmar el CLA y debes asignar el issue correspondiente en el PR, antes que comencemos a revisar tu PR. Saludos!

@jaov jaov changed the title Traducido observables.md a español Traducido observables.md a español fix #214 Oct 6, 2020
Arreglados algunos errores ortográficos y de redacción en la traducción.
Copy link
Member

@gustavguez gustavguez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hola @jaov, deben de incluir el archivo original con la extensión en.md para poder revisar tu PR. saludos!

@japhethCalzadaL
Copy link

@jaov sigues trabajando con el archivo? me gustaría contribuir

@jaov
Copy link
Author

jaov commented May 10, 2021

El archivo se encuentra en esta URL dentro del proyecto. ¿Es necesario que lo suba aparte?

@dalejodc
Copy link
Member

Si @jaov, el archivo original se debe de incluir con extensión .en.md. En este caso el PR debe de incluir el archivo observables.md (el que está actualmente en el PR) y el archivo observables.en.md que es el archivo original en inglés (el que hace falta en el PR).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants